Skip to content

test: Improve worktree command regression tests#12

Merged
wasabeef merged 1 commit intomainfrom
refactor-test
Apr 11, 2026
Merged

test: Improve worktree command regression tests#12
wasabeef merged 1 commit intomainfrom
refactor-test

Conversation

@wasabeef
Copy link
Copy Markdown
Owner

Summary

Improve the regression test suite around worktree command flows before larger code refactors.

What Changed

  • added create-flow regression coverage for tag-based worktree creation and switch file output
  • added config discovery regression coverage for bare and linked worktree contexts
  • replaced several low-signal unit tests with higher-signal command contract tests for switch, rename, and delete
  • made new command tests use unique worktree names so they remain stable in full-suite runs
  • removed placeholder utils tests that were not asserting behavior

Why

The existing suite had good breadth, but some tests were mostly checking trivial string properties while key command entry points were still lightly protected. This change strengthens the safety net so upcoming refactors can rely on behavior-focused coverage.

Impact

  • improves confidence when refactoring command-layer code
  • reduces false confidence from low-value tests
  • keeps the full suite green while adding stronger command regressions

Validation

  • cargo test --all-features -- --test-threads=1

@wasabeef wasabeef changed the title [codex] Improve worktree command regression tests test: Improve worktree command regression tests Apr 11, 2026
@github-actions
Copy link
Copy Markdown

Code Metrics Report

Coverage Test Execution Time
69.5% 2m9s

Reported by octocov

@wasabeef wasabeef marked this pull request as ready for review April 11, 2026 03:14
@wasabeef wasabeef merged commit 81023b0 into main Apr 11, 2026
6 checks passed
@wasabeef wasabeef deleted the refactor-test branch April 11, 2026 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant